-
Notifications
You must be signed in to change notification settings - Fork 88
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Design rationale for DHT #304
Merged
taxmeifyoucan
merged 2 commits into
eth-protocol-fellows:main
from
chirag-parmar:dht-rationale
Jul 24, 2024
Merged
Design rationale for DHT #304
taxmeifyoucan
merged 2 commits into
eth-protocol-fellows:main
from
chirag-parmar:dht-rationale
Jul 24, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks for such a quick dive into this!
Co-authored-by: Mário Havel <[email protected]>
taxmeifyoucan
merged commit Jul 24, 2024
fdf8443
into
eth-protocol-fellows:main
1 of 2 checks passed
taxmeifyoucan
added a commit
that referenced
this pull request
Jul 24, 2024
* Add study group info (#298) * ✨ feat: Prehistory of Ethereum (#226) * ✨ feat: p2p computing * ✨ feat: Overview: Bitcoin * ✨ feat: Overview: Ethereum world comptuer * ✨ feat: Overview: Applications * 🥢 nit: * nit Co-authored-by: Mário Havel <[email protected]> * ✨ feat: Revised intro * ✨ feat: Internet * ✨ feat: Map of ARPANET * ✨ feat: Unix * ✨ feat: fin * 🥢 nit: intro * ✨ feat: Documentary GNU/Linux, Crypto Anarchy --------- Co-authored-by: rahul <[email protected]> Co-authored-by: Mário Havel <[email protected]> * Add files via upload (#302) * Create content for scourge under the staking economics track (#300) * change title lowercase to uppercase * create content for scourge under the staking economics track * remove extra space in reference * add words to wordlist * reordered the numbering and removed unsed content * MEV-track and staking economics track in the same table removed the added tablea header: MEV-track and staking economics track can be in the same table * reordered the staking economics table content --------- Co-authored-by: DanGoron <[email protected]> * Design rationale for DHT (#304) * design rationale for DHT * Apply suggestions from code review Co-authored-by: Mário Havel <[email protected]> --------- Co-authored-by: Mário Havel <[email protected]> --------- Co-authored-by: rahul <[email protected]> Co-authored-by: rahul <[email protected]> Co-authored-by: Rory Arredondo <[email protected]> Co-authored-by: Glory Agatevure <[email protected]> Co-authored-by: DanGoron <[email protected]> Co-authored-by: Chirag Parmar <[email protected]>
The spell check is still broken because of the words never included in wordlist |
@shyam-patel-kira thanks for pointing it out. I did the typo check but forgot to add to the list. Should I create a new PR? |
taxmeifyoucan
added a commit
that referenced
this pull request
Aug 27, 2024
* Add study group info (#298) * ✨ feat: Prehistory of Ethereum (#226) * ✨ feat: p2p computing * ✨ feat: Overview: Bitcoin * ✨ feat: Overview: Ethereum world comptuer * ✨ feat: Overview: Applications * 🥢 nit: * nit Co-authored-by: Mário Havel <[email protected]> * ✨ feat: Revised intro * ✨ feat: Internet * ✨ feat: Map of ARPANET * ✨ feat: Unix * ✨ feat: fin * 🥢 nit: intro * ✨ feat: Documentary GNU/Linux, Crypto Anarchy --------- Co-authored-by: rahul <[email protected]> Co-authored-by: Mário Havel <[email protected]> * Add files via upload (#302) * Create content for scourge under the staking economics track (#300) * change title lowercase to uppercase * create content for scourge under the staking economics track * remove extra space in reference * add words to wordlist * reordered the numbering and removed unsed content * MEV-track and staking economics track in the same table removed the added tablea header: MEV-track and staking economics track can be in the same table * reordered the staking economics table content --------- Co-authored-by: DanGoron <[email protected]> * Design rationale for DHT (#304) * design rationale for DHT * Apply suggestions from code review Co-authored-by: Mário Havel <[email protected]> --------- Co-authored-by: Mário Havel <[email protected]> * Meeting #5 notes (#307) * ✨ feat: Meeting #2 notes * 🥢 nit: link * 🥢 nit: * update notes 2 * Notes for wiki contributors meeting #3 * Formatting * Spell check * ✨ feat: Meeting 4 notes by Rory * ✨ feat: Meeting notes 5, closes #299 --------- Co-authored-by: rahul <[email protected]> Co-authored-by: Mário Havel <[email protected]> Co-authored-by: Siddharth Vaderaa <[email protected]> * Consensus Layer Wiki Page (#246) * CL init, Update overview of CL * Update ordering * Add validators section to overview * add iamges; added Beacon chain explainer; checkpoints and finality; slots and epochs * Added validator life cylce * fix typo; update wordlist * Update state of validators * Improve flow of the page;added simpler explanations; added some links * add introduction; minor fixes * Minor typos ffix * add introduction; added byzantine generals problem * revamp overview structure * fix dark background in svg * complete overview of CL; added cl-architecture structure * Added Blocktree and fork-choice rules * fix some typos; update wordlist * add reorgs and reversion * Add liveness and safey comparision * Add some more details on consensus protocol * Add architecture and blobs * stf; control flow * fix a broken link; added gasper file * Use consistent naming for PoW and PoS * Complete cl-architecture * Update structure of cl-networking * fix typos; added words to wordlist * remove whitespace * address some nits * Omit some redudant content * remove redundant content; fix broken links * Update proposer and validator set wording Co-authored-by: Mário Havel <[email protected]> * Address some more nits * Add resources; omit whitespace * fix typos * Update wordlist * nit: grammar Co-authored-by: rahul <[email protected]> * nit: space Co-authored-by: rahul <[email protected]> * nit: word Co-authored-by: rahul <[email protected]> * nit: word Co-authored-by: rahul <[email protected]> * clean up Co-authored-by: rahul <[email protected]> * nit: spell Co-authored-by: rahul <[email protected]> * nit: content captilization * Update wordlist * nit: diagram name Co-authored-by: rahul <[email protected]> * nit: word Co-authored-by: rahul <[email protected]> * nit: clean up Co-authored-by: rahul <[email protected]> * nit: clean up Co-authored-by: rahul <[email protected]> * nit: title Co-authored-by: rahul <[email protected]> * nit: wording Co-authored-by: rahul <[email protected]> * nit: title Co-authored-by: rahul <[email protected]> * fix flow for the transition * Update beacon-api.md --------- Co-authored-by: Mário Havel <[email protected]> Co-authored-by: rahul <[email protected]> * add links to mirrors (#308) closes #297 * Update wordlist.txt * Modified EVM Resources section (#311) * fixed type * enhanced EVM wiki * enhanced EVM wiki * fixed titles * fixed titles --------- Co-authored-by: rahul <[email protected]> Co-authored-by: rahul <[email protected]> Co-authored-by: Rory Arredondo <[email protected]> Co-authored-by: Glory Agatevure <[email protected]> Co-authored-by: DanGoron <[email protected]> Co-authored-by: Chirag Parmar <[email protected]> Co-authored-by: Siddharth Vaderaa <[email protected]> Co-authored-by: kira <[email protected]> Co-authored-by: Zaryab <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR adds a section (under
protocol/design-rationale.md
) which addresses the use of DHTs within the Ethereum protocol. It also aims to answer, though not directly, the question raised here.